Skip to content

fix: pin react-test-renderer version to fix npm install issues #471

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
jokerttu merged 3 commits into from
Sep 11, 2025
Merged

fix: pin react-test-renderer version to fix npm install issues #471

jokerttu merged 3 commits into from
Sep 11, 2025

Conversation

@jokerttu
Copy link
Contributor

@jokerttu jokerttu commented Sep 11, 2025

  • Pins react and react-test-renderer versions to 19.1.0
  • Fixes eslint config to old eslint version to fix npm dependency issues.
  • Fixes lint issues.

Pre-launch Checklist

  • I read the Contributor Guide and followed the process outlined there for submitting PRs.
  • I signed the CLA.
  • I listed at least one issue that this PR fixes in the description above.
  • I updated/added relevant documentation
  • I added new tests to check the change I am making
  • All existing and new tests are passing.

@jokerttu jokerttu requested a review from caio1985 September 11, 2025 12:41
@jokerttu jokerttu changed the title fix: pin react version to fix npm install issues fix: pin react-test-renderer version to fix npm install issues Sep 11, 2025
caio1985
caio1985 approved these changes Sep 11, 2025
View reviewed changes
Copy link
Member

@caio1985 caio1985 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@jokerttu jokerttu merged commit e3f0f81 into main Sep 11, 2025
12 checks passed
@jokerttu jokerttu deleted the fix/npm-install-issues branch September 11, 2025 13:12
@googlemaps-bot googlemaps-bot mentioned this pull request Sep 11, 2025
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@caio1985 caio1985 caio1985 approved these changes

Assignees

@ArturoSalazarB16 ArturoSalazarB16

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@jokerttu @caio1985 @ArturoSalazarB16